soc: silabs: Move Kconfig symbols for HAL selection to HAL #97853
+378
−363
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
asmellby
force-pushed
the
feature/silabs-kconfig-cleanup
branch
2 times, most recently
from
962fb2a to
aeda5c3
Compare
asmellby
requested review from
Martinhoff-maker,
jerome-pouiller and
jhedberg
zephyrbot
requested review from
HaavardRei,
HoZHel,
PavelVPV,
Thalley,
alwa-nordic,
bjarki-andreasen,
cvinayak,
hermabe,
nordic-krch,
rettichschnidi,
sjanc and
yonsch
jhedberg
previously approved these changes
Oct 19, 2025
asmellby
force-pushed
the
feature/silabs-kconfig-cleanup
branch
from
aeda5c3 to
80bf05b
Compare
jhedberg
previously approved these changes
Oct 19, 2025
Remove DCDC defconfigs from Series 2 boards. These have no effect, Series 2 derives DCDC configuration from devicetree. Signed-off-by: Aksel Skauge Mellbye <[email protected]>
Sort symbol selections for Series 2 SoCs. Signed-off-by: Aksel Skauge Mellbye <[email protected]>
Kconfig symbols for selecting HAL content should be part of the HAL module integration, not defined in the SoC tree. Define the sleeptimer symbol for WiSeConnect and SiSDK since both use it. In the future, WiSeConnect should include the SiSDK configuration and reuse it instead of redefining everything itself. This is a larger scale refactor that this commit doesn't start tackling. Signed-off-by: Aksel Skauge Mellbye <[email protected]>
The indirection through a backend symbol for PM implementation isn't necessary. Define symbol for PM HAL in HAL Kconfig, and leverage it at SoC level. Signed-off-by: Aksel Skauge Mellbye <[email protected]>
Kconfig symbols for selecting HAL content should be part of the HAL module integration, not defined in the SoC tree. Signed-off-by: Aksel Skauge Mellbye <[email protected]>
COUNTER_GECKO_STIMER is defined by the counter driver. It should not be present in SoC Kconfig. Signed-off-by: Aksel Skauge Mellbye <[email protected]>
CRYPTO_ACC_GECKO_TRNG only applies to Series 2. Don't define it at the top level. Signed-off-by: Aksel Skauge Mellbye <[email protected]>
Move the Kconfig symbol for the SE HAL to hal_silabs. Select the symbol in the entropy driver rather than unconditionally at the SoC level. Signed-off-by: Aksel Skauge Mellbye <[email protected]>
Series 2 specific defconfigs for Bluetooth related options should be set in the Series 2 specific defconfig file. Signed-off-by: Aksel Skauge Mellbye <[email protected]>
asmellby
force-pushed
the
feature/silabs-kconfig-cleanup
branch
from
80bf05b to
4e2b8e5
Compare
Martinhoff-maker
previously approved these changes
Oct 20, 2025
jhedberg
previously approved these changes
Oct 20, 2025
jerome-pouiller
previously approved these changes
Oct 20, 2025
The Kconfig symbols for selecting HAL content should be part of the HAL module integration, not defined by the SoC. Split the symbols between the Series 0/1 Gecko HAL and Series 2 SiSDK HAL when moving them. For now, the Series 0/1 HAL symbols retain their name, while new names consistent with the symbols already defined in the module integration layer are used for the Series 2 HAL. Signed-off-by: Aksel Skauge Mellbye <[email protected]>
asmellby
dismissed stale reviews from jerome-pouiller, jhedberg, and Martinhoff-maker
via
d628ab1
asmellby
force-pushed
the
feature/silabs-kconfig-cleanup
branch
from
4e2b8e5 to
d628ab1
Compare
Martinhoff-maker
approved these changes
Oct 20, 2025
|
jerome-pouiller
approved these changes
Oct 20, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The Kconfig symbols for selecting HAL content should be part
of the HAL module integration, not defined by the SoC. Split the
symbols between the Series 0/1 Gecko HAL and Series 2 SiSDK HAL
when moving them.
For now, the Series 0/1 HAL symbols retain their name, while new
names consistent with the symbols already defined in the module
integration layer are used for the Series 2 HAL.